5. Code quality: logging, constants, exceptions, protocols, architecture docs#10
Open
musicalplatypus wants to merge 6 commits into
Open
5. Code quality: logging, constants, exceptions, protocols, architecture docs#10musicalplatypus wants to merge 6 commits into
musicalplatypus wants to merge 6 commits into
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ve_paths() Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add TRAINING_BACKEND_TINYML_TINYVERSE, DATA_DIR_CLASSES, DATA_DIR_FILES, DATA_DIR_IMAGES constants. Use existing TRAINING_DEVICE_CUDA in params.py defaults instead of bare 'cuda' strings. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…nyml-modelmaker Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…er, etc.) Define typing.Protocol classes that formalize the implicit contracts already followed by ModelRunner, ModelTraining, ModelCompilation, and DatasetHandling. Uses structural subtyping so no existing classes need modification. Enables static type checking and documents the interface contracts for future implementations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ment analysis Documents the full tinyml-tensorlab architecture including all four sub-repos, pipeline flow, configuration system, model/quantization/NAS subsystems, a Mermaid architecture diagram, and 12 design/implementation improvement recommendations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adithya-Thonse
pushed a commit
that referenced
this pull request
Jun 12, 2026
Merge in TINYML-ALGO/tinyml-agent-skills from 2026/pranav_a to main * commit 'be83fc6267a898cc6eeab38ffe0c0a344528ab97': minor fixes removed assets, included autoMP quant
Adithya-Thonse
added a commit
that referenced
this pull request
Jun 12, 2026
de8af16d Pull request #45: https://jira.itg.ti.com/browse/TINYML_ALGO-698 REVERT: e48ef1a Pull request #14: TINYML_ALGO-711: fixing readme REVERT: 16fc6a6 TINYML_ALGO-711: fixing readme REVERT: e3639d2 Pull request #13: removing pycache REVERT: f8bb3b7 removing pycache REVERT: dd38428 Pull request #12: restructuring agent skill REVERT: ff02a0e restructuring agent skill REVERT: d26c6a5 Pull request #11: fixing tiny ml name REVERT: 640ffd3 fixing tiny ml name REVERT: 4ee3a19 Pull request #10: 2026/pranav a REVERT: be83fc6 minor fixes REVERT: e3a5700 removed assets, included autoMP quant REVERT: 1af575a Pull request #9: correcting npu devices list REVERT: 31e9eb1 correcting npu devices list REVERT: 59b209b Pull request #8: improving readme REVERT: 8c3260b improving readme REVERT: 668916f Pull request #7: improving readme REVERT: 68686b3 improving readme REVERT: 814316e Pull request #6: fixes to readme and marketplace json REVERT: e4bc0b4 fixes to readme and marketplace json REVERT: 6a64208 Pull request #5: fixes to readme REVERT: 0f9c868 fixes to readme REVERT: 52f95ff Pull request #4: 2026/pranav a REVERT: 443295d fixes to readme REVERT: 1881112 fixes to readme and marketplace json REVERT: 229ab57 Pull request #3: 2026/pranav a REVERT: 6519104 minor readme fix REVERT: 38e9f9f minor readme fix REVERT: db81f81 Pull request #2: minor readme fix REVERT: 1c0737a minor readme fix REVERT: 0a0c02d Pull request #1: minor readme fix REVERT: b682335 minor readme fix REVERT: 062eb39 Initial Commit git-subtree-dir: tinyml-agent-skills git-subtree-split: de8af16d9e23de3e9bda3d811a0ebdece1178260
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A collection of code quality improvements that make the codebase more maintainable, debuggable, and self-documenting. No behavioral changes — all improvements are structural.
Changes
1. Replace
print()withloggingmoduleprint()calls intinyml-modelmakerto use Python'sloggingmodule2. Extract duplicated path resolution
timeseries/runner.pyandvision/runner.pyinto sharedresolve_paths()utility3. Replace magic strings with named constants
TRAINING_DEVICE_CPU,TRAINING_DEVICE_CUDA,TRAINING_DEVICE_MPSconstants'cpu','cuda','tinyml_tinyverse'with constants4. Replace
assert/sys.exit/raise stringwith proper exceptionsassertstatements used for input validation →raise ValueErrorsys.exit()calls →raise RuntimeError5. Protocol definitions for component interfaces
protocols.pywithtyping.Protocoldefinitions forModelRunner,Trainer,DatasetLoader6. Architecture documentation
ARCHITECTURE.mdwith codebase structure diagram, component descriptions, and improvement analysisFiles Changed (22 files)
All within
tinyml-modelmakerTesting